CI: simplify onboard scene workflows - #1682
Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughThe A2A3 CI workflows remove the ChangesA2A3 SDMA CI workflow
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
0bdc20a to
fa0e7cb
Compare
c577a93 to
c0b0cea
Compare
c0b0cea to
1881cd3
Compare
Summary
ci.ymlandci-self-cpu.ymluse the marker-based a2a3 SDMA splitst-onboard-*jobs instead of splitting them into extra jobsmaintopology while documenting the onboard DFX behaviorRelation to #1639
#1639's main dedupe is already present on
main:ci.ymlandci-self-cpu.ymlshare the canonical_detect-changes.yml, common job bodies are reusableworkflow_callworkflows, and the pip cache / venv scaffolding lives in composite actions.This PR keeps that direction but limits itself to the remaining a2a3 onboard cleanup: remove the legacy SDMA path mode, keep the marker-based split, and document that the onboard DFX smokes stay inside the base onboard jobs rather than becoming separate CI jobs.
Checkout intentionally remains inline as
actions/checkout@v5inside the reusable workflows. A local composite action cannot safely wrap the first checkout step because local actions are loaded from the workspace after checkout has already happened.Validation
main(c9ccaf65on August 6, 2026)git diff --check